Skip to content

Cache partial responses #91

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Feb 10, 2025
Merged

Cache partial responses #91

merged 11 commits into from
Feb 10, 2025

Conversation

BoD
Copy link
Collaborator

@BoD BoD commented Jan 30, 2025

Implements #57.

  • Add a partial mode to CacheBatchReader that sets an Error as the value of fields that are a cache miss instead of throwing

  • When using the bundled FetchPolicy interceptors, partial results are still treated as full cache misses with an exception. Use a custom interceptor to see cache partial responses.

@BoD BoD requested a review from martinbonnin as a code owner January 30, 2025 13:51
@BoD BoD force-pushed the partial-responses branch from 37ae199 to 008b48c Compare February 5, 2025 15:43
…terceptors still treat partial results as a cache miss
@BoD BoD force-pushed the partial-responses branch from d9034fd to 6e7befd Compare February 7, 2025 19:09
@BoD BoD merged commit b88dfa5 into main Feb 10, 2025
2 checks passed
@BoD BoD deleted the partial-responses branch February 10, 2025 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants